Skip to content

Conversation

@eilifhl
Copy link
Contributor

@eilifhl eilifhl commented Feb 11, 2026

image

Trenger kanskje hjelp med tekst og subteksten/hintene

@eilifhl eilifhl self-assigned this Feb 11, 2026
@eilifhl eilifhl marked this pull request as ready for review February 11, 2026 09:52
@eilifhl eilifhl marked this pull request as draft February 11, 2026 11:40
@robines
Copy link
Member

robines commented Feb 11, 2026

Veldig fint!!

Kan hende det blir for mange klikk med "fortsett".

Enig, dropp "fortsett". Og da burde du endre de sirklene også, kanskje endre de til piler, og flytt de til høyre for teksten (se bilde for ish hvordan jeg tenker). Kanskje også prøv å gi undertekstene en litt tynnere font weight.

image

@eilifhl eilifhl marked this pull request as ready for review February 11, 2026 18:43
@eilifhl eilifhl added the frontend Task related to frontend. label Feb 11, 2026
<h1 className={styles.headerTitle}>Hvordan vil du logge inn?</h1>

<div className={styles.picker}>
<button type="button" className={styles.choiceWrapper} onClick={() => handleChoice('new')}>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bruk en anchor tag her, ikke button. Mye bedre accessibility og så kan du droppe handleChoice

Tilbake
</button>

<nav aria-label="Velg innlogging" className={styles.picker}>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hvorfor ble bruken av nav fjernet? Vil si den er veldig relevant å bruke her

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oops

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend Task related to frontend.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants